Skip to content

Conversation

@eemeli
Copy link
Collaborator

@eemeli eemeli commented Feb 17, 2025

Digit size option values MAY have an implementation-defined upper limit, but let's make sure that does not exclude anything allowed by the digit-size-option rule for literal values.

@eemeli eemeli added functions Issue pertains to the default function set Agenda+ Requested for upcoming teleconference LDML47 labels Feb 17, 2025
Implementations MAY define an upper limit on the _resolved value_
of a digit size option option consistent with that implementation's practical limits.
of a digit size option option consistent with that implementation's practical limits,
as long as all integer values from 0 to 99 (inclusive) are supported.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

99 is way beyond the significant digits supported by an IEEE754 double. If we are going to require minimal values, it shouldn't be more than needed with double, which means it may vary according to the option.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that this isn't the right fix. I think what you're trying to say is maybe:

Suggested change
as long as all integer values from 0 to 99 (inclusive) are supported.
of a digit size option consistent with that implementation's practical limits.
Implementations that do so MUST emit an _Unsupported Operation_ error.
The literal values for integers from 0 to 99 (inclusive) MUST NOT produce
a _Bad Option_ error.

(Note that your text has "option option")

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My intent here was to clarify the general case, but arguably that does complicate matters; the current language allows more clearly for an implementation to define a different upper bound for each digit size option.

The "option option" text is currently in the spec; we should fix that separately if discarding this change.

@aphillips
Copy link
Member

closing per 2025-02-17 call discussion. typo to be fixed in 1007

@aphillips aphillips closed this Feb 17, 2025
@aphillips aphillips deleted the luftballons branch February 17, 2025 18:06
@eemeli eemeli added this to the LDML 47 milestone Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agenda+ Requested for upcoming teleconference functions Issue pertains to the default function set

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants